home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c++-part2 / 18438 < prev    next >
Encoding:
Text File  |  1996-08-05  |  2.0 KB  |  56 lines
  1. Path: news.ruhr-uni-bochum.de!usenet
  2. From: Karsten Soete <Karsten.Soete@rz.ruhr-uni-bochum.de>
  3. Newsgroups: comp.lang.c++
  4. Subject: Re: Return
  5. Date: 20 Apr 1996 14:43:50 GMT
  6. Organization: Ruhr-Universitaet Bochum, Rechenzentrum
  7. Message-ID: <4lat76$hh3@sun168.rz.ruhr-uni-bochum.de>
  8. NNTP-Posting-Host: dialslip-39.rz.ruhr-uni-bochum.de
  9.  
  10. morrisd@nevada.edu (DAMON MORRIS) schreibt:
  11. > I am having a really bad problem with some return statements...
  12. > I declare a function to return an int value then in the function
  13. > I check to see if two strings are equal and if so then I use another
  14. > variable to see what to return...
  15. > 
  16. > int charisma(int ability, char *choice)
  17. >  {
  18. >   if (choice == "Loyalty Base")
  19. >    {
  20. >     if (ability==10) return 20;
  21. >    }
  22. >  }
  23. > 
  24. > This little program will never work for some reason. When I call it
  25. > like this charisma(10,"Loyalty Base") it always returns some number
  26. > that it shouldn't. The number that is returned is usually between
  27. > 5000-10000. If I wrote the value of choice to the screen before it
  28. > checked to see if it was equal to "Loyalty Base" it always says that
  29. > it is equal to "Loyalty Base", yet it never gets to the second if.
  30. > I have no idea what is the matter...I will send anyone full source if
  31. > needed to fix the problem, but the source is 1000+ lines....
  32. > 
  33. > 
  34.  
  35. First the equal-operator (==) must be implented for char *.
  36. This is the case only for string-classes and I think it doesn't
  37. apply here. To compare to strings conventionally use strcmp.
  38.  
  39. I your case a simple pointer-comparation between choice and the
  40. memory-location of "Loayalty Base" is done. This won't work.
  41.  
  42. So the comparation never matches and the return-value in the
  43. if-statement can't be returned.
  44. By the way, what then. Your function doesn't define a return-value
  45. in case the comparation won't match. 
  46.  
  47. Try this code:
  48. #include <string.h>
  49.  
  50. int charisma(int ability, char *choice){
  51.   if(!strcmp(choice,"Loyalty Base"))
  52.     if(ability==10) return 20;
  53.  
  54.   return 10; //in case comparations don't match.
  55. }
  56.